-
Notifications
You must be signed in to change notification settings - Fork 230
feat(arc): add ClickBench results for Arc on c6a.4xlarge #634
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are going to push a new update of this PR in a few minutes. Thank you for marking the issues.
…nd c6a.4xlarge in aws
This comment was marked as resolved.
This comment was marked as resolved.
Just updated the files and make it public the repo. Thanks. |
|
||
# Install Python and dependencies | ||
echo "Installing dependencies..." | ||
pip3 install fastapi uvicorn duckdb pyarrow requests gunicorn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This requires running pip
with --break-system-packages
.
Would it be possible to create a Python venv? See e.g. chdb/benchmark.sh for an example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, we have in our start.sh in the repo, I'm adding to this script.
|
||
# Create API token for benchmark | ||
python3 << EOF | ||
from api.auth import AuthManager, Permission |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got the next error here:
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
ImportError: cannot import name 'Permission' from 'api.auth' (/data/ClickBench/arc/arc/api/auth.py)
I checked, there is indeed no Permission
class in file auth.py
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uff, thank you for this, its old code, in our repo we have this right. Let me update it here too.
Hey everyone,
We’re the new folks in the neighborhood, sharing ClickBench results for Arc, our time-series warehouse that’s launching soon.
I’ve made sure everything follows the benchmark requirements, but happy to adjust if needed.
Appreciate your work on this project!
– Ignacio